Skip to content

fix(deps): skip the local Maven reactor parent - #1579

Merged
FelixTJDietrich merged 1 commit into
mainfrom
1570-ignore-local-maven-parent
Aug 29, 2026
Merged

fix(deps): skip the local Maven reactor parent#1579
FelixTJDietrich merged 1 commit into
mainfrom
1570-ignore-local-maven-parent

Conversation

@FelixTJDietrich

@FelixTJDietrich FelixTJDietrich commented Aug 29, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Description

Stop Renovate from querying Maven Central for de.tum.cit.aet:hephaestus-server-parent. That coordinate is the unpublished parent of this repository's own Maven reactor, so no external registry can provide an update for it.

The last hosted Renovate run successfully extracted 382 dependencies but aborted after Maven Central rate-limited this uncached lookup. This narrow rule removes the repository-controlled failure path without disabling Maven updates or suppressing warnings for real dependencies.

Follow-up to #1570 and Dependency Dashboard #709.

How to test

  • bun run format:config
  • bun run check
  • npx --yes --package renovate renovate-config-validator --strict renovate.json
  • Run Renovate 44.50.3 locally in Maven-only, full dry-run mode and confirm both extracted occurrences report Dependency: de.tum.cit.aet:hephaestus-server-parent, is disabled, while Maven package lookup completes.
  • After merge and a cooldown for Maven Central's rate limit, trigger the hosted Renovate job and confirm it reaches update and dashboard reconciliation.

Checklist

  • No changeset is required because this only changes dependency-bot discovery and has no shipped behavior.
  • Real Maven dependencies, plugins, wrappers, and external parents remain enabled.
  • This does not hide external-host failures for any registry-backed dependency.

Summary by CodeRabbit

  • Chores
    • Disabled automated dependency updates for the Hephaestus server parent Maven package.

@FelixTJDietrich
FelixTJDietrich requested a review from a team as a code owner August 29, 2026 09:04
@coderabbitai

coderabbitai Bot commented Aug 29, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 24f53f18-3891-4ebd-8eb1-db60994d73b8

📥 Commits

Reviewing files that changed from the base of the PR and between 32bdfb1 and 61ad31c.

📒 Files selected for processing (1)
  • renovate.json

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

Walkthrough

The Renovate configuration adds a package rule that disables updates for the Maven package de.tum.cit.aet:hephaestus-server-parent.

Changes

Renovate configuration

Layer / File(s) Summary
Disable Maven parent updates
renovate.json		 A packageRules entry disables Renovate updates for the specified Maven package.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 61ad3

This narrowly prevents dependency tooling from querying an unpublished local Maven parent while leaving other dependency updates enabled; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: disabling Renovate updates for the local Maven reactor parent.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 1570-ignore-local-maven-parent

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@FelixTJDietrich
FelixTJDietrich merged commit 8f787c6 into main Aug 29, 2026
28 of 30 checks passed
@FelixTJDietrich
FelixTJDietrich deleted the 1570-ignore-local-maven-parent branch August 29, 2026 09:08
@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

📚 Documentation Preview

Preview has been removed (PR closed)

@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

🧩 Storybook Preview

Preview has been removed (PR closed)

@github-project-automation github-project-automation Bot moved this to Backlog in Hephaestus Sep 1, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Hephaestus Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@FelixTJDietrich FelixTJDietrich

Labels

None yet

Projects

Hephaestus
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@FelixTJDietrich